feat: Implement persistent, focused root directory for sidebar with '..' navigation#136
feat: Implement persistent, focused root directory for sidebar with '..' navigation#136msavox wants to merge 3 commits into
Conversation
|
Thank you very much for your work! Root directory switching through double-clicking is a great idea! In terms of specific implementation, there might still be some details that need to be determined, such as: 1. After setting a certain directory A as the root by double-clicking, there's no way to click the directory tree to switch exactly under this directory, you can only jump to its subdirectory or parent directory. 2. Due to historical reasons, the related code is quite chaotic, which caused a bug after your modifications; I still need to check this carefully. Additionally, please do not git commit unrelated files such as project.pbxproj and Main.storyboard. |
|
Yeah the implementation is not the best, I vibe coded it (it was my first
time compiling an app for macOS) just to show you the feature I would like
to have.
Your app is really great, very lightweight, simple and neat. I really
appreciate your work.
Feel free to reimplement the functionality as you prefer (maybe you can
also add in the preferences a filed to select the root directory as an
alternative).
Thank you for your work, I was looking for something to replace digiKam on
macOS!
Il mar 27 gen 2026, 07:25 netdcy ***@***.***> ha scritto:
… *netdcy* left a comment (netdcy/FlowVision#136)
<#136 (comment)>
Thank you very much for your work! Root directory switching through
double-clicking is a great idea!
In terms of specific implementation, there might still be some details
that need to be determined, such as: 1. After setting a certain directory A
as the root by double-clicking, there's no way to click the directory tree
to switch exactly under this directory, you can only jump to its
subdirectory or parent directory. 2. Due to historical reasons, the related
code is quite chaotic, which caused a bug after your modifications; I still
need to check this carefully.
Additionally, please do not git commit unrelated files such as
project.pbxproj and Main.storyboard.
—
Reply to this email directly, view it on GitHub
<#136 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BBZWJSVWY5PX3LQXIDWSBNL4I4AHJAVCNFSM6AAAAACR5UEWV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTQMBTGM2DQMBTHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
Hi netdcy, Just wanted to update you on the latest changes in this Pull Request. Following your feedback:
The branch has been updated with these changes, and I believe it's now ready for another review. Thank you for your time and guidance! |
|
Thank you again for your efforts! However, there are still quite a few issues with the code implementation, and relying solely on LLM isn’t sufficient. The main reason is that my implementation of the directory tree is very messy; I suspect no one but me would be able to make changes to it. Additionally, this commit includes files such as .gitignore, project.pbxproj, Main.storyboard, and pull_request_description.txt, which suggests that you may not be fully familiar with git yet. Therefore, I’m afraid to say that this pull request is still a long way from being ready for merging. But overall, it’s a great idea. If you don’t mind, I plan to rewrite this feature myself in the future. |
2 participants
This Pull Request introduces a new feature to enhance sidebar management, providing more granular control over directory viewing and improving user navigation.
Key Changes:
..): If the current root directory is not the filesystem's root, a special ".." entry is added at the beginning of the list of children for the current root folder. Double-clicking ".." will navigate up one level in the directory hierarchy.Motivation:
These changes significantly improve usability for users working with deep repositories or folder structures, allowing them to focus the directory tree on a relevant part of the filesystem and easily restore that context on each launch.